Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[xpcc] Separate the rx handling part of the update fct of the dispatcher #970

Merged

Conversation

se-bi
Copy link
Contributor

@se-bi se-bi commented Mar 11, 2023

Call that separated function instead
and have it available as a public function as well.

Then it is possible to use the XPCC dispatcher and postman,
and combine e.g. the zeromq gateway example with that.

Copy link
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure!

@se-bi se-bi force-pushed the feature/separate-xpcc-update-rx-handling branch from e5ce603 to 17d5aa7 Compare March 12, 2023 15:43
@se-bi se-bi force-pushed the feature/separate-xpcc-update-rx-handling branch from 17d5aa7 to 711cb80 Compare March 12, 2023 15:49
@salkinium
Copy link
Member

salkinium commented Mar 13, 2023

Could you rebase (after #843 is merged), I forgot to merge this before the previous PR 🤦

Call that separated function instead
and have it available as a public function as well.
@se-bi se-bi force-pushed the feature/separate-xpcc-update-rx-handling branch from 711cb80 to 2273bae Compare March 14, 2023 12:25
@se-bi
Copy link
Contributor Author

se-bi commented Mar 14, 2023

Could you rebase (after #843 is merged), I forgot to merge this before the previous PR 🤦

Sure, done.

@salkinium salkinium merged commit 2273bae into modm-io:develop Mar 14, 2023
@se-bi se-bi deleted the feature/separate-xpcc-update-rx-handling branch March 15, 2023 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants